Skip to content

fix: include all relevant dormitory info in grouped answers(생활관 미지정 답변의 생활관별 정보 누락 개선)#34

Merged
1024andrew merged 1 commit intodevfrom
fix/#33
May 4, 2026
Merged

Conversation

@1024andrew
Copy link
Copy Markdown
Contributor

@1024andrew 1024andrew commented May 4, 2026

유형

  • Feat
  • Fix
  • Design
  • Docs
  • Chore
  • Hotfix

수정 사항

  • 생활관을 특정하지 않은 질문에서 여러 생활관 정보가 검색된 경우, 확인된 생활관별 정보를 누락하지 않도록 답변 프롬프트를 보강했습니다.
  • 특정 생활관에 해당 시설이 없다는 정보가 참고 정보에 포함된 경우, 해당 내용도 함께 안내하도록 개선했습니다.

수정 배경

dormitory = null 상태에서 다리미 있어? 질문 시 검색 결과에는 제1학생생활관, 제2학생생활관, 제3학생생활관 관련 정보가 포함되어 있었지만, 답변은 제2학생생활관 정보만 안내하는 문제가 있었습니다.

검색 결과 자체는 정상적으로 들어왔기 때문에, 검색 개수를 늘리기보다 답변 생성 단계에서 여러 생활관 정보를 누락하지 않도록 프롬프트를 보강했습니다.

테스트

  • 다리미 있어? + dormitory = null

기대 효과

  • 생활관 미지정 질문에서 확인된 생활관별 정보를 더 빠짐없이 안내
  • 특정 생활관에 시설이 없는 경우도 함께 안내
  • 불필요하게 검색 top_k를 늘리지 않고 답변 품질 개선

스크린샷

종합프로젝트 챗봇 개선3 image

@1024andrew
Copy link
Copy Markdown
Contributor Author

프롬프트만 고친거라 그냥 머지했습니다

@1024andrew 1024andrew merged commit 2573724 into dev May 4, 2026
1 check passed
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the prompt instructions in app/services/generator.py to ensure that information for multiple dormitories is comprehensively reported, including cases where specific facilities are missing. A minor whitespace change was also introduced in app/services/chat_service.py. Feedback suggests that these new instructions may conflict with an existing directive to be concise and avoid forced distinctions, recommending that the existing instruction be updated to ensure clarity when dormitory information differs.

Comment thread app/services/generator.py
Comment on lines +67 to +68
질문에서 생활관을 특정하지 않았고 참고 정보에 여러 생활관의 정보가 포함되어 있으면, 확인된 생활관별 정보를 누락하지 말고 함께 안내해라.
특정 생활관에 해당 시설이 없다는 정보가 참고 정보에 있으면 그 내용도 함께 안내해라.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

새롭게 추가된 지침들이 기존 69번 라인의 지침("생활관별 구분을 강제로 만들지 말고... 간결하게 답변해라")과 상충될 우려가 있습니다. 69번 지침은 모델이 정보를 하나로 합치거나 요약하게 만들어, 이번 수정의 목적인 '생활관별 정보의 상세한 안내'를 방해할 수 있습니다. 69번 라인도 함께 수정하여, 정보가 상이할 경우 명확히 구분하여 답변하도록 지시하는 것이 좋습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant